Skip to content

feat: implement manual type for async index build #7104

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 9 commits into
base: main
Choose a base branch
from
Draft

feat: implement manual type for async index build #7104

wants to merge 9 commits into from

Conversation

@SNC123
Copy link
Contributor

@SNC123 SNC123 commented Oct 16, 2025

I hereby agree to the terms of the GreptimeDB CLA.

#6756 (Tracking Issue)

What's changed and what's your intention?

Implemented manual build type for async index build:

  • New Admin Function: A new administrative function, build_index(table_name), is added to initiate a rebuild for a specific table.
  • Targeted Rebuild: For efficiency, this operation identifies and rebuilds only the indexes that are inconsistent, rather than all of them.

PR Checklist

Please convert it to a draft if some of the following conditions are not met.

  • I have written the necessary rustdoc comments.
  • I have added the necessary unit tests and integration tests.
  • This PR requires documentation updates.
  • API changes are backward compatible.
  • Schema or data changes are backward compatible.

SNC123 added 9 commits October 13, 2025 18:25
@SNC123
feat: impl four types index build
4d3854d 
Signed-off-by: SNC123 <[email protected]>
@SNC123
test: add tests for four types index build
4b02e86 
Signed-off-by: SNC123 <[email protected]>
@SNC123
test: add sqlness test for manual index build
f13ee77 
Signed-off-by: SNC123 <[email protected]>
@SNC123
fix: add region request support and correct sqlness
f376667 
Signed-off-by: SNC123 <[email protected]>
@SNC123
fix: update cargo.toml for proto and resolve conflicts
f35d914 
Signed-off-by: SNC123 <[email protected]>
@SNC123
fix: rebase
cbb8a12 
Signed-off-by: SNC123 <[email protected]>
@SNC123
chore: clippy
a52bd01 
Signed-off-by: SNC123 <[email protected]>
@SNC123
fix: toml fmt and correct sqlness
1d5d088 
Signed-off-by: SNC123 <[email protected]>
@SNC123
fix: correct sqlness result
f125b26 
Signed-off-by: SNC123 <[email protected]>
@github-actions github-actions bot added docs-not-required This change does not impact docs. size/L labels Oct 16, 2025
This was referenced Oct 16, 2025
Open
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@waynexia waynexia Awaiting requested review from waynexia waynexia will be requested when the pull request is marked ready for review waynexia is a code owner

@v0y4g3r v0y4g3r Awaiting requested review from v0y4g3r v0y4g3r will be requested when the pull request is marked ready for review v0y4g3r is a code owner

@evenyag evenyag Awaiting requested review from evenyag evenyag will be requested when the pull request is marked ready for review evenyag is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

docs-not-required This change does not impact docs. size/L

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@SNC123